Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Readme Content Verification #3

Merged
merged 1 commit into from
Feb 26, 2019

Conversation

scbedd
Copy link
Member

@scbedd scbedd commented Feb 22, 2019

  • Cleaning up namespace to just pieces we want public. Getting rid of import *
  • Adding verification of a readme content.
    • Currently works by looking at the section headers.
    • Supports regex matching of said section headers
    • Works with the omitted_paths configuration so we will ignore readmes as expected

Before merge, I will update the readme with additional details on example usage of the readme verification.

Issue
@Azure/azure-sdk-eng

…port ordering cleanup and whitespace cleanup. updating all copyright comments with the appropriate boilerplate comment from opensource. readme updates for content verification usage
@scbedd scbedd force-pushed the readme-content-verification branch from 0739737 to 19b1712 Compare February 26, 2019 17:16
@scbedd scbedd merged commit a2ff91a into Azure:master Feb 26, 2019
@mmacy
Copy link
Contributor

mmacy commented Mar 1, 2019

@scbedd Hey Scott, just wanted to verify the tool's validation is based on the latest README format defined in README-TEMPLATE.md.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants